(Mobile) UI improvements #705
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
We can also merge UI changes step by step in multiple PRs :) |
traines-source
force-pushed
the
mobile-ui
branch
from
e5304b1 to
17a8d8b
Compare
traines-source
force-pushed
the
mobile-ui
branch
from
17a8d8b to
eaa2053
Compare
|
Then have a look :) I might also squash some commits, then you can merge without squashing everything to keep a better/more detailed history. |
felixguendling
approved these changes
Jan 19, 2025
| } = $props(); | ||
| </script> | ||
|
|
||
| <div class="m-4 text-red-600"><div class="min-w-0">{e}</div></div> |
There was a problem hiding this comment.
Suggested change
| <div class="m-4 text-red-600"><div class="min-w-0">{e}</div></div> | |
| <div class="m-4 text-destructive"><div class="min-w-0">{e}</div></div> |
or maybe introduce a error theme dependent error color?
I would not say it has no benefit but IMO it's not worth the time invest to rebase and squash. This time is better invested in code improvements, new features, bug fixes, etc. :) |
This was referenced Jan 21, 2025
Merged
D3vZro
added a commit
to MoViDe-Project/motis
that referenced
this pull request
Jan 30, 2025
* Use `maxTravelTime` to filter results (motis-project#668) * Use 'maxTravelTime' to filter results * Change unit to minutes * Fix formatting * Fix type for MSVC build * Update nigiri dependency * Set limit for maximum travel time * GBFS: Performance/Memory Improvements + Vehicle Types (motis-project#670) * gbfs: partition providers, faster updates, less memory usage * fix vehicle_docks_available * slightly faster geofencing zone mapping * rename provider_cache -> provider_file_infos * ts formatting * formatting * clang fixes * clang fix * clang fixes * trying to fix apple clang * static_cast all the things * review changes * fix include * 65k gbfs providers should be enough * remove obsolete comment * sort vehicle status before diff * fill cache during gbfs update * cleanup * partition vehicle types by form factor + propulsion type * rental api changes (RENTAL mode + form factor + propulsion type) * ui fix * rename provider segment -> products * vehicle type id -> idx * rename more segment -> products * one more r-tree to speed up geofencing zone mapping * clang fix * fix api descriptions * api: add rental provider filter * ui: fix direct connection display for rental connections * return constraint support, allow roundtrip for direct connections * ui formatting * fix missing gbfs data after update * http proxy support * share decompressed bitvecs between routing requests * fix missing initializer --------- Co-authored-by: Felix Gündling <[email protected]> * replace wheelchair with pedestrianProfile and useRoutedTransfers params (motis-project#683) * replace wheelchair with pedestrianProfile and useRoutedTransfers parameters * fix test * ui: debounce search requests * increase trains limit to support big cities like Paris/London/.. * add timeout option * cli changes (motis-project#685) * cli changes * wip * update nigiri * Update README.md: not beta anymore * update nigiri: speedup ~8% by not finding 24h+ transfers * basic benchmark and QA tooling (motis-project#686) * bench * wip * wip * wip * wip * fix command line flag handling * full dataset test (motis-project#687) * full dataset test * wip * wip * wip * wip * wip * wip * wip * wip * wip * wip * body size limit 128M * cmake: Add option to pass flags not used for host tools (motis-project#689) * update nigiri: rt update fixes time travel * more rebostness against invalid delay update data motis-project/nigiri#162 * update nigiri: trim stop time before empty check for interpolation * motis config: fix crash with no parameters * update nigiri: better trip names (motis-project#695) * polish translation * Detailed transfers flag + output exactly what nigiri routed (motis-project#698) * wip * output exactly what nigiri routed * remove logging output * remove unused variable * fix eval * wip * wip * Initial style guide docs/STYLE.md * Fix missing closing tag (motis-project#700) * Fix missing closing tag * Add sections for important tools This adds a section for `strong`, as well as `vector_map` and `vecvec`. * Fix index out of range for stop_times last arrival of multi-section run (motis-project#704) * fix index out of range for stop_times last arrival of multi-section run * follow style guide Co-authored-by: Felix Gündling <[email protected]> --------- Co-authored-by: Felix Gündling <[email protected]> * (Mobile) UI improvements (motis-project#705) * ui: move components to lib * ui: make components more responsive * ui: more compact layout * ui: i18n fixes * ui: make toggle button state more visible * ui: error handling, avoid repeated effect triggering * ui: more small screen adjustments * ui: url state handling * ui: show backend error msgs, compact transfer display * UI fixes (motis-project#710) * ui: fall back to black routeTextColor (gtfs spec) * update ui deps * ui: workaround to avoid clickthrough * ui: linting * update nigiri: monotonicity enforcement on utc times (fixes interpolation for frequency expanded trips) * ui changes * Improve formatDuration (motis-project#713) * Update formatDuration.ts * fix formatting * fix formatting * Fix formatting * osr_footpaths: add missing footpaths (motis-project#707) * osr_footpaths: add missing footpaths * update osr (ramp support) * wip * wip * wip * wip * wip * wip * wip * wip * wip * wip * formatting --------- Co-authored-by: Michael Kutzner <[email protected]> Co-authored-by: Pablo Hoch <[email protected]> Co-authored-by: Felix Gündling <[email protected]> Co-authored-by: Felix Gündling <[email protected]> Co-authored-by: Jonah Brüchert <[email protected]> Co-authored-by: Traines <[email protected]> Co-authored-by: Altonss <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Some improvements for the UI, without changing too much (see public-transport/transitous#789). This is not perfect yet, consider it a WIP.
pushStatelike in prima-mobile, which means less refactoring and easier web server configuration)TODO: